Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add support crsf baro altitude combined vario sensor #9438

Merged
merged 6 commits into from
Dec 26, 2023
Merged

Add support crsf baro altitude combined vario sensor #9438

merged 6 commits into from
Dec 26, 2023

Conversation

r1000ru
Copy link
Contributor

@r1000ru r1000ru commented Nov 2, 2023

In some cases (absence of a GNSS receiver or spoofing), it is impossible to find out the altitude in telemetry. The crsf protocol has a telemetry packet 0x09 for baro altitude, which can also include its rate of change (vario).

I suggest adding this package, and if barometer is used, do not use the vario packet, since the transmission of one packet is faster than two.

@DzikuVx DzikuVx added this to the 7.1 milestone Nov 3, 2023
@DzikuVx DzikuVx changed the base branch from master to release_7.1.0 December 9, 2023 12:58
@DzikuVx DzikuVx merged commit 5d77d07 into iNavFlight:release_7.1.0 Dec 26, 2023
14 checks passed
@b14ckyy b14ckyy mentioned this pull request Mar 20, 2024
Closed
DzikuVx added a commit that referenced this pull request Mar 26, 2024
@DzikuVx
Revert "Merge pull request #9438 from dronecontrol-ru/baro_alt_vario"
85ec0a9 
This reverts commit 5d77d07, reversing
changes made to 3b218b6.
@DzikuVx DzikuVx mentioned this pull request Mar 26, 2024
Merged
@Target0815 Target0815 mentioned this pull request Apr 9, 2024
Closed
JulianTiller pushed a commit to JulianTiller/inav that referenced this pull request Oct 18, 2024
@DzikuVx @JulianTiller
Revert "Merge pull request iNavFlight#9438 from dronecontrol-ru/baro_…
a163bd3 
…alt_vario"

This reverts commit 5d77d07, reversing
changes made to 3b218b6.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
7.1
Development

Successfully merging this pull request may close these issues.
3 participants
@r1000ru @DzikuVx @r1000